Support read-only publish service #11367
Conversation
|
casbin这个库对权限管理是否有帮助？ |
目前没有多用户, 只需要基于角色的访问控制, 感觉用不太上👀 |
|
@88250
|
|
收到，打算放到特性版本 v3.1.0 中合并。 |
顺便检查一下 API 的鉴权中间件设置的是否合理, 因为没有逐项检查, 可能有不合理的地方 |
|
还有就是前端的设置面板与该功能相关的设置 API 可能需要之后由 @Vanessa219 实现一下 |
|
初步看了下，还缺失配置部分和用户指南文档，你后面有空的话先把这两部分补齐吧。 |
|
|
收到，改得有点多，审核会比较慢，谢谢。 |
88250
changed the title
|
这两个地方要引用一下新增的文档（3个用户指南都要）：
|
那我先暂不解决合并冲突了, 因为出现得太频繁了, 等合并前再集中解决 |
|
明天发布 v3.0.16，发布后如果没有严重问题，后面的版本就是 v3.1.0 了。 |
review 完成后记得戳我一下, 我解决一下冲突 |
|
好的，还需要一久。 |
日语的用户指南也要添加了 |
|
我这边已经 review 了后端 go 代码，前端等 @Vanessa219 看看 |
| fontFamilyHTML += `<option value="${item}"${window.siyuan.config.editor.fontFamily === item ? " selected" : ""}>${item}</option>`; | ||
| }); | ||
| fontFamilyElement.innerHTML = fontFamilyHTML; | ||
| if (response.code === 0) { |
There was a problem hiding this comment.
是会返回其他大于0的值么？会的话需写一下相关处理，不会的话就不用添加 if 了。
| @@ -504,8 +504,9 @@ export const exportMd = (id: string) => { | |||
| }); | |||
| }); | |||
| return; | |||
| } else if (response.code === 0) { | |||
| showMessage(window.siyuan.languages.exportTplSucc); | |||
There was a problem hiding this comment.
是会返回其他大于1的值么？会的话需写一下相关处理，不会的话就不用添加 if 了。
| @@ -150,8 +150,10 @@ export const initAssets = () => { | |||
| return; | |||
| } | |||
| } | |||
| window.siyuan.config.appearance = response.data.appearance; | |||
| loadAssets(response.data.appearance); | |||
| if (response.code === 0) { | |||
There was a problem hiding this comment.
是会返回其他大于0的值么？会的话需写一下相关处理，不会的话就不用添加 if 了。以免其他 code 漏了处理。
|
@Zuoqiu-Yingyi 有空的话麻烦看下问题，我们争取尽早合并，ღ( ´･ᴗ･` )比心 |
已合并最新更改并处理冲突 |
|
@Zuoqiu-Yingyi 那 3 个 review 你好像还没回复 |
啊，不是等待 reviewer 处理的状态嘛👀？ |
|
V 姐不是提了 3 个返回值处理的问题嘛，应该是让你来解决 |
解决方案：无需处理 |
|
但是回复呢😂 |
|
不需要处理的话移除即可。 |
这里的不需要处理是指不需要对错误进行额外的处理, 原来的回调没有兼容请求失败的情况，移除后会抛出异常 |
|
这样是不是所有请求都要进行修改，或者在 fetch 中进行改进更合理。 |
|
感谢贡献！先合并了，后面再调整。 |
|
体验了一下，很不错的功能。如果能指定哪些笔记本可以暴露就更好了。免得管理多个工作空间，麻烦。 |
|
不能直接发布某一篇笔记吗？ |
|
这个功能很好，能不能只发布一篇笔记，能不能增加禁止复制的功能 |
REL: #11364
添加设置项 | Add setting items
{ "publish": { "enable": true, "port": 6808, "auth": { "enable": true, "accounts": [ { "username": "1", "password": "1", "memo": "这是用户 1" }, { "username": "2", "password": "2", "memo": "这是用户 2" } ] } } }siyuan.config.publish.enable: 是否开启发布服务siyuan.config.publish.port: 自定义发布服务端口, 设置为 0 则使用随机端口siyuan.config.publish.auth.enable: 发布服务是否启用 BasicAuth 认证siyuan.config.publish.auth.accounts: 发布服务 BasicAuth 认证账户列表已经过测试 | TESTED